Add DigiDollar integration guide across digibyte.org#164
Conversation
|
preview: hero: documentation section: footer: |
Adds links to the new DigiDollar_integration_guide.pdf in three places on every locale index.html (root + 35 locale folders): - Hero CTA pill next to the existing 'Integration guide' pill - New slide in the 'DigiByte Documents & Guides' slick slider, between the Integration guide and Media sheet slides, using images/digibyte_symbol_06c.svg as the avatar - New 'DGB DigiDollar Guide' entry in the footer 'Resources' list, directly under 'DGB Integration Guide' Labels are English across all locales. The en/ redirect-only stub is unchanged.
DennisPitallano
force-pushed
the
add-digidollar-integration-guide
branch
from
48a7134 to
5226e1b
Compare
DGBNOOB
left a comment
DGBNOOB
left a comment
There was a problem hiding this comment.
The commit description for 5226e1b made the review process very efficient. The PR modifies 37 files in total.
Scope of Review
I reviewed the changes for:
“Add DigiDollar integration guide across site”
The commit description accurately reflects what the PR implements:
What the PR Adds
Across root + 35 locale folders, each index.html now includes:
-
Hero CTA pill
A new DigiDollar guide pill placed next to the existing Integration Guide pill. -
New slide in the “DigiByte Documents & Guides” slick slider
Inserted between the Integration Guide and Media Sheet slides, using
images/digibyte_symbol_06c.svgas the avatar. -
Footer Resources entry
A new “DGB DigiDollar Guide” link added directly under DGB Integration Guide.
Localization Notes
- All labels remain English across all locales, as stated.
- The
en/redirect-only stub is unchanged, which matches the commit description.
Files Reviewed
I reviewed all 37 modified files, including:
af, ar, bg, cs, da, de, el, en-us, es, fa, fi, fil, fr, hi, hr, hu, id, it, ja, ms, nb, nl, pl, pt-br, pt, ro, ru, sl, sq, sv, sw, th, tr, vi, zh, index.html
Plus the binary PDF:
docs/DigiDollar_integration_guide.pdf
(located atDGBio/docs/DigiDollar_integration_guide.pdfin the source repo)
Conclusion
All changes described in the commit message are present, correctly implemented, and consistently applied across every locale. While I didn’t run or test the changes, I reviewed all 37 modified files and confirmed that each one includes the DigiDollar integration guide additions in the specific locations described in the commit message.
Recommend approval cACK
ycagel
left a comment
ycagel
left a comment
There was a problem hiding this comment.
cACK. Great work @DennisPitallano!
3 participants
Closes #163
Summary
Adds a link to the new
docs/DigiDollar_integration_guide.pdfin three places across every localeindex.html(root + 35 locale folders, 36 files total), mirroring the existing "Integration guide" pattern.What's added per page
images/digibyte_symbol_06c.svgas the avatar.<li>("DGB DigiDollar Guide") directly under "DGB Integration Guide".Asset
docs/DigiDollar_integration_guide.pdfimages/digibyte_symbol_06c.svg(existing DigiByte symbol, no new image asset)Notes
en/index.htmlJS-only redirect stub is intentionally not modified.ar/andfa/use<div class="testimonials__slide rtl">for the slider tile;bg/index.htmlhas the herohome-content__rightblock uncommented (all others are commented out).Verification
Grep counts across the 36 modified files:
DigiDollar_integration_guide.pdfreferencesdigibyte_symbol_06c.svg(slider avatar)DGB DigiDollar Guide(footer li)DigiDollar guide(case-insensitive, incl. alt text)Out of scope